Claude finished @engijlr's task in 4m 23s —— View job

PR Review

Target: origin/v17/dev · Based on commit: cc5c743e

Adds responsive CSS and a burger-menu sidebar toggle to the backoffice so the Log Viewer (and section sidebar) is usable on viewports narrower than 920 px. No desktop layout is affected.

• Modified public API: UmbSplitPanelElement — new sidebarOpen reflected boolean property/attribute; UmbBackofficeContext — new mobileSidebarOpen observable, toggleMobileSidebar(), setMobileSidebarOpen(); @umbraco-cms/backoffice/const — new export UMB_MOBILE_BREAKPOINT
• Breaking changes: None — all changes are additive.

Suggestions

• src/Umbraco.Web.UI.Client/src/packages/core/components/split-panel/split-panel.element.ts:53: sidebarOpen is the only @property() on this element without a JSDoc comment. The three existing properties (snap, lock, position) each have a /** ... */ doc block. Add one here to keep the public API surface documented.

• src/Umbraco.Web.UI.Client/src/packages/core/components/split-panel/split-panel.element.ts:327-331 and 348-351: The slot[name='end'] flex rule (flex: 1 1 auto; min-width: 0) appears twice — once at root level and again inside the @media block. The root rule already applies on mobile, so the media-query repetition is redundant.

  /* root — already covers mobile */
  slot[name='end'] {
      flex: 1 1 auto;
      min-width: 0;
  }
  
  /* inside @media — the :host([connected]) wrapping doesn't change the flex behaviour */
  :host([connected]) slot[name='end'] {
      flex: 1 1 auto;   /* ← same values, can be removed */
      min-width: 0;
  }

• src/Umbraco.Web.UI.Client/src/packages/log-viewer/workspace/views/overview/log-overview-view.element.ts:179: Inside the mobile @media block, #common-messages-container is given grid-area: common-messages again — but the parent switched to display: flex, so grid-area has no effect. Only the --uui-box-default-padding override matters here; the grid-area line can be dropped.

• src/Umbraco.Web.UI.Client/src/packages/core/section/default/default-section.element.ts (mobile UX): The sidebar overlay currently closes on navigation (changestate) and on viewport resize back to desktop. There is no click-outside or swipe-away handler. For a first iteration this is reasonable, but tapping the page content while the sidebar is open leaves it open with no escape other than the burger button. Worth a follow-up.

Approved with Suggestions for improvement

The implementation is solid: clean CSS-only responsive breakpoints for the log viewer, a well-structured context-driven sidebar toggle, and proper cleanup in connectedCallback/disconnectedCallback. The suggestions above are minor polish items — none need to block merge.